Skip to content

Conversation

@silver-eunjoo
Copy link
Collaborator

🛰️ Issue Number

🪐 작업 내용

1. 방장 레디 풀림 이슈

  • 제가 모든 플레이어의 레디 상태를 원상복구 시켜놔서 발생했던 이슈입니다.
  • resetAllPlayerReadyStates(); 메서드를 이용해서 방장을 제외한 플레이어의 레디 상태만 바꾸도록 변경했습니다.

2. PLAYER_LIST 메세지 누락 이슈

  • 연결이 끊긴 플레이어가 있을 때만 PLAYER_LIST가 브로드캐스팅되도록 되어있었습니다.
  • disconnectedPlayers 리스트가 비었을 때만 해당 로직을 타고, 아닐 때는 PLAYER_LIST를 바로 브로드캐스팅해주도록 분기 처리했습니다.

3. PLAYER_LIST 브로드캐스팅 여러번 되는 이슈

  • 연결이 끊긴 플레이어가 있을 때, SYSTEM_NOTICE, PLAYER_LIST를 여러 번 브로드캐스팅하게끔 구현되어있었습니다.
  • SYSTEM_NOTICE는 개별 플레이어가 나갈 때마다 브로드캐스팅해주는 게 맞지만, PLAYER_LIST는 한 번만 응답으로 내려가도 되기 때문에, 반복문 밖으로 뺐습니다.

📚 Reference

✅ Check List

  • 코드가 정상적으로 컴파일되나요?
  • 테스트 코드를 통과했나요?
  • merge할 브랜치의 위치를 확인했나요?
  • Label을 지정했나요?

@silver-eunjoo silver-eunjoo self-assigned this Jul 27, 2025
@silver-eunjoo silver-eunjoo added the bug Something isn't working label Jul 27, 2025
Copy link
Collaborator

@LimKangHyun LimKangHyun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

고생하셨습니다!

}

messageSender.sendBroadcast(
destination, MessageType.PLAYER_LIST, toPlayerListResponse(room));
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

SystemNotice 와 플레이어 리스트는 동기화가 되어야하지 않을까합니다!

Copy link
Collaborator

@sehee123 sehee123 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수고하셨습니다!

@silver-eunjoo silver-eunjoo merged commit e607059 into dev Jul 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[fix] 게임 종료 시 PLAYER_LIST 응답 누락, 방장 레디 풀리는 이슈 해결

6 participants